Skip to content

GPII-2046: Ensure that settings handlers always write to active profiles#477

Open
javihernandez wants to merge 2 commits into
GPII:masterfrom
javihernandez:GPII-2046
Open

GPII-2046: Ensure that settings handlers always write to active profiles#477
javihernandez wants to merge 2 commits into
GPII:masterfrom
javihernandez:GPII-2046

Conversation

@javihernandez

Copy link
Copy Markdown
Member

See https://issues.gpii.net/browse/GPII-2046.
Work in progress - not ready to be merged.

@gpii-bot

gpii-bot commented Oct 5, 2016

Copy link
Copy Markdown

CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details.

@gpii-bot

gpii-bot commented Oct 5, 2016

Copy link
Copy Markdown

CI job passed.

1 similar comment
@gpii-bot

Copy link
Copy Markdown

CI job passed.

"settingsHandlers": {
"configuration": {
"type": "gpii.orca",
"options": {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there no corresponding linux pull request to this? What does the settings handler do if the user token is not supplied?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, there's no linux side for this. As I mentioned back in the meeting we talked about this thing I removed this constraint in Orca settings handler during the last update but I forgot to remove the universal "bits" of it. I agree that this can be ready to go in as a separate pull request since we still need to make some accommodations in UNIVERSAL before the real issue (GPII-2046) can be addressed. Let me know if you want me to make a new JIRA/pr and get this merged ASAP.

@gtirloni

gtirloni commented Feb 3, 2017

Copy link
Copy Markdown
Contributor

ok to test

@gpii-bot

gpii-bot commented Feb 3, 2017

Copy link
Copy Markdown

CI job passed.

@kaspermarkus

Copy link
Copy Markdown
Member

Will pick up again when a higher priority solution needs this functionality

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants